-
Notifications
You must be signed in to change notification settings - Fork 1
feat/107-add-advanced-mode-per-question #131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat/107-add-advanced-mode-per-question #131
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements per-question advanced mode functionality, replacing the global advanced mode toggle with individual advanced toggles for each question. The global "advanced mode" switch now serves as the default setting for newly created questions. Each question can independently enable/disable advanced features (images, explanations, and answer images) through its own toggle.
Key Changes
- Refactored advanced mode from global quiz-level to per-question level with an
advancedboolean property - Modified
sanitizeQuestionsto use per-question advanced flags instead of a global parameter - Added per-question advanced mode toggle switch in the question form UI
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| src/components/quiz/quiz-editor.tsx | Extends Question type with optional advanced property; refactors sanitizeQuestions to use per-question flags; updates initialization logic to detect and set advanced mode per question; updates global advanced mode label to clarify it's the default for new questions |
| src/components/quiz/question-form.tsx | Adds per-question advanced toggle switch; replaces advancedMode prop with question.advanced property; duplicates multiple-choice checkbox in non-advanced view for better UX |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
24683fd to
0cf189c
Compare
Antoni-Czaplicki
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Antoni-Czaplicki
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

No description provided.